New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update for new web push #205

Open

tranvanhieu01012002 wants to merge 4 commits into laravel-notification-channels:master from tranvanhieu01012002:update-for-new-web-push

+74 −35

tranvanhieu01012002 commented Jul 11, 2024

Currently browser support newSubscription, so I create new PR to handle new keys for web push.
new web push

Task:

Update migrate file -> remove old key, update to new keys, add option UUID
Add option UUID on document
Update send method on src/WebPushChannel.php to allow insert by new key
Update test script

tranvanhieu01012002 added 4 commits

July 12, 2024 00:03


          feat: update migrate table

74e0692


          feat: update for new keys web push


          feat: update new keys for test

c133da5


          fix: format issues

4f4a389

Author

tranvanhieu01012002 commented Jul 22, 2024

@cretueusebiu can you check my PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet